Skip to content

Conversation

@dorimedini-starkware
Copy link
Collaborator

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator Author

dorimedini-starkware commented Apr 15, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link

github-actions bot commented Apr 15, 2025

Artifacts upload workflows:

@dorimedini-starkware dorimedini-starkware self-assigned this Apr 15, 2025
@dorimedini-starkware dorimedini-starkware marked this pull request as ready for review April 15, 2025 14:16
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 83d6363 to 1dbb071 Compare April 15, 2025 14:31
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from e325490 to 728954b Compare April 15, 2025 14:31
@dorimedini-starkware dorimedini-starkware marked this pull request as draft April 15, 2025 14:33
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 1dbb071 to d9d9e5f Compare April 15, 2025 14:56
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 728954b to 8fe7b87 Compare April 15, 2025 14:56
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from d9d9e5f to 8688f92 Compare April 15, 2025 15:09
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 8fe7b87 to a1fd18f Compare April 15, 2025 15:09
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 8688f92 to 4061106 Compare April 15, 2025 15:49
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from a1fd18f to 866c853 Compare April 15, 2025 15:49
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 4061106 to 97825d7 Compare April 15, 2025 16:15
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 866c853 to af6a8b7 Compare April 15, 2025 16:15
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 97825d7 to 2e6df53 Compare April 15, 2025 16:32
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from af6a8b7 to 2d9c368 Compare April 15, 2025 16:33
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_committer_and_os_cli_add_command_to_dump_the_os_compiled_program_to_file branch from 2e6df53 to da89616 Compare April 16, 2025 07:31
@dorimedini-starkware dorimedini-starkware marked this pull request as ready for review June 3, 2025 07:46
@dorimedini-starkware dorimedini-starkware force-pushed the 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli branch from 0ae3e91 to 0e13d04 Compare June 3, 2025 07:48
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 4098fbb to 8cc3b3f Compare June 3, 2025 07:48
Copy link
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: 4 of 5 files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware and @TzahiTaub)


a discussion (no related file):

Previously, TzahiTaub (Tzahi) wrote…

Pease update

new py side

@dorimedini-starkware dorimedini-starkware force-pushed the 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli branch from 0e13d04 to 8558852 Compare June 3, 2025 09:22
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 8cc3b3f to 27e92bb Compare June 3, 2025 09:22
Copy link
Contributor

@TzahiTaub TzahiTaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware and @nimrod-starkware)


a discussion (no related file):

Previously, dorimedini-starkware wrote…

new py side

Waiting for it to pass

Copy link
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware and @TzahiTaub)


a discussion (no related file):

Previously, TzahiTaub (Tzahi) wrote…

Waiting for it to pass

will require some changes (TODO cleanup logic is causing string mismatches, I think). will push a fix to this pr first; PTAL at the two preceding PRs for now (5945, 5946)

@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 27e92bb to dca637c Compare June 3, 2025 12:00
@dorimedini-starkware dorimedini-starkware force-pushed the 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli branch from 8558852 to 9f22c4b Compare June 3, 2025 15:32
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from dca637c to 6ce1da8 Compare June 3, 2025 15:32
Copy link
Contributor

@TzahiTaub TzahiTaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r5, 1 of 1 files at r6, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

Copy link
Contributor

@TzahiTaub TzahiTaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli branch from 9f22c4b to 03b37e8 Compare June 4, 2025 14:10
@dorimedini-starkware dorimedini-starkware force-pushed the 04-14-feat_starknet_os_remove_the_program_from_the_os_runner_inputs branch from 6ce1da8 to 394fb8f Compare June 4, 2025 14:11
Copy link
Contributor

@TzahiTaub TzahiTaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r7, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli branch from 03b37e8 to 6161f7d Compare June 5, 2025 11:35
@dorimedini-starkware dorimedini-starkware changed the base branch from 04-19-feat_apollo_starknet_os_program_add_test_contracts_add_dumper_in_cli to main-v0.14.0 June 5, 2025 12:18
Copy link
Collaborator Author

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 4 of 5 files at r3, 1 of 1 files at r5, 1 of 1 files at r7, 7 of 7 files at r8, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)

@dorimedini-starkware dorimedini-starkware added this pull request to the merge queue Jun 5, 2025
Merged via the queue into main-v0.14.0 with commit ebf4b56 Jun 5, 2025
44 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants